Skip to content

Disable git text conversion via .gitattributes, remove CI workarounds#187

Merged
emlowe merged 1 commit intoChia-Network:mainfrom
altendky:disable-git-text-conversion
Mar 4, 2026
Merged

Disable git text conversion via .gitattributes, remove CI workarounds#187
emlowe merged 1 commit intoChia-Network:mainfrom
altendky:disable-git-text-conversion

Conversation

@altendky
Copy link
Contributor

Summary

  • Add * -text to .gitattributes to disable git's automatic line-ending conversion for all files
  • Remove the now-redundant git config --global core.autocrlf false CI step from precommit.yml

Context

Same change as Chia-Network/chia-blockchain#20514. The .gitattributes attribute takes precedence over core.autocrlf, so the CI workaround step is no longer needed — and the protection extends to all contributors' local environments too, not just CI.

Add `* -text` to .gitattributes to prevent git from performing any
line-ending conversion on any file. This makes the per-workflow
`git config --global core.autocrlf false` step in precommit
redundant, so remove it.
@altendky altendky closed this Mar 3, 2026
@altendky altendky reopened this Mar 3, 2026
@altendky
Copy link
Contributor Author

altendky commented Mar 3, 2026

https://github.com/Chia-Network/chia-dev-tools/actions/runs/22199207546/job/65476430517 was a wonky error. and i forgot that close and reopen would trigger run approval. sorry...

@emlowe
Copy link
Contributor

emlowe commented Mar 4, 2026

The pre-commit error is a bad cache afaict - if you update your branch from main, I added a comment to the precommit yaml to force a cache miss and this resolves the issue in other PRs

Well maybe that isnt helping - somehow the pre-commit action cache is getting poisoned - I don't think there is a flag to turn it off though

@emlowe
Copy link
Contributor

emlowe commented Mar 4, 2026

Ah, i think maybe it's a runner problem

@emlowe emlowe closed this Mar 4, 2026
@emlowe emlowe reopened this Mar 4, 2026
@emlowe emlowe merged commit c577092 into Chia-Network:main Mar 4, 2026
69 of 73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants